Add color map to DirectoryItemInput so we can give different color by equipmentType and add Fields needed for contingency list by filter #839
Conversation
… equipment type and add new Fields for Aleas by filter Signed-off-by: basseche <[email protected]>
basseche
changed the title
Signed-off-by: basseche <[email protected]>
Signed-off-by: basseche <[email protected]>
basseche
force-pushed
the
contingencyListByFilter
branch
3 times, most recently
from
b0f47df to
5b94394
Compare
Signed-off-by: basseche <[email protected]>
basseche
force-pushed
the
contingencyListByFilter
branch
from
5b94394 to
bf01977
Compare
basseche
changed the title
|
| fields: elements, | ||
| append, | ||
| remove, | ||
| } = useFieldArray({ |
There was a problem hiding this comment.
À mon avis il n'y a plus besoin de renommer fields ici vu qu'il est déjà renommé en bas, autant garder le nom d'origine ici plutôt que de le changer 2 fois
| {elementsWithMetaData.map((item, index) => ( | ||
| <Box key={`Box${item.id}`} sx={{ display: 'flex', flexDirection: 'column', gap: 1 }}> | ||
| <Chip | ||
| key={item.id} |
There was a problem hiding this comment.
Il y a toujours besoin de la prop key vu qu'il a été ajouté sur la box au-dessus ?
| item.specificMetadata?.equipmentType && | ||
| equipmentColorsMap?.has(item.specificMetadata?.equipmentType) | ||
| ? equipmentColorsMap.get(item.specificMetadata.equipmentType) | ||
| : undefined, |
There was a problem hiding this comment.
| item.specificMetadata?.equipmentType && | |
| equipmentColorsMap?.has(item.specificMetadata?.equipmentType) | |
| ? equipmentColorsMap.get(item.specificMetadata.equipmentType) | |
| : undefined, | |
| equipmentColorsMap?.get(item.specificMetadata?.equipmentType) | |
Corrigez moi si je me trompe mais je crois que du coup ça revient exactement a la même chose
| } | ||
| /> | ||
| {elementsWithMetaData.map((item, index) => ( | ||
| <Box key={`Box${item.id}`} sx={{ display: 'flex', flexDirection: 'column', gap: 1 }}> |
There was a problem hiding this comment.
C'est pas quelque chose dont on risque d'avoir besoin ailleurs ? il faudrait peut-être sortir ce code et en faire un composant ?
| <Chip | ||
| key={item.id} | ||
| size="small" | ||
| sx={{ |
There was a problem hiding this comment.
Au lieu d'utiliser sx et la variable css backgroundcolor, pas possible d'utiliser tout simplement color de Chip ?
J'ai pas testé mais à mon avis ça doit être la même chose.
https://mui.com/material-ui/api/chip/#props
| } | ||
| /> | ||
| {elementsWithMetaData.map((item, index) => ( | ||
| <Box key={`Box${item.id}`} sx={{ display: 'flex', flexDirection: 'column', gap: 1 }}> |
There was a problem hiding this comment.
Petite question, pourquoi mettre box devant l'id ?
Je pense que c'est moins performant que de mettre directement l'id.
| } = useFieldArray({ | ||
| name, | ||
| }); | ||
| const elementsWithMetaData: TreeViewFinderNodeProps[] = elements as unknown as TreeViewFinderNodeProps[]; |
There was a problem hiding this comment.
On est sûr que les champs correspondent toujours à ce type TreeViewFinderNodeProps ?
J'ai essayé de remonter pour voir où se composant était utilisé et franchement j'ai toujours aucune idée d'à quoi ressemble ce formulaire.
En tout cas rien que le nom est bizarre, d'après le nom c'est censé être utilisé uniquement dans l'arbre mais je vois qu'il est un peu partout...
| return ''; | ||
| } | ||
| return equipmentType !== EquipmentType.HVDC_LINE | ||
| ? (BASE_EQUIPMENTS[equipmentType as EquipmentType]?.label ?? '') |
There was a problem hiding this comment.
Pourquoi ne pas rajouter ce type d'équipement dans BASE_EQUIPMENTS ?
No description provided.